-
Notifications
You must be signed in to change notification settings - Fork 125
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mimic joints independent of the combination of command interfaces #111
Mimic joints independent of the combination of command interfaces #111
Conversation
Transmissions are coming in the very near future. Can you take a look
whether that format would work for you?
ros-controls/ros2_control#633
|
Thanks for this hint, great to see it coming :) |
As far as parsing the functionality should already be there. It just
doesn't load transmissions for you. You probably can even do without that PR
…On Tue, 15 Feb 2022, 20:19 Christoph Fröhlich, ***@***.***> wrote:
Transmissions are coming in the very near future. Can you take a look
whether that format would work for you? ros-controls/ros2_control#633
<ros-controls/ros2_control#633>
Thanks for this hint, great to see it coming :)
My real hardware has a mechanically linked telescope arm (one actuator,
two prismatic links). From the PR alone I do not get if this is supported
yet, but I'll give it a try within the next days..
—
Reply to this email directly, view it on GitHub
<#111 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA24PYMUK7HHA5M5HFHXECTU3KYOHANCNFSM5OKX54HQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@ahcorde It seems that my extension of the mimic joints does not meet with approval... Should I rewrite it by adding the offset only, and adding a paragraph to the readme discussing the limits of the mimicked joints? |
@christophfroehlich since we are adding more parameters for mimic joints (rightly so!) I would propose that we update our parser on that. The proposal would be to implement this syntax (@bmagyar what do you think?) <joint name="right_finger_joint">
<command_interface name="effort"/>
<state_interface name="position"/>
<state_interface name="velocity"/>
<state_interface name="effort"/>
</joint>
<joint name="left_finger_joint">
<mimic joint="right_finger_joint" multiplier="1" offset="0" />
<command_interface name="position"/>
<state_interface name="position"/>
<state_interface name="velocity"/>
<state_interface name="effort"/>
</joint> This would make the syntax the same as for the rest of URDF. The changes are fairly simple to do on ros2_control repository. If you would like to contribute, I can give you exact pointers what should be done. I don't have time unfortunately to implement this, but can support you with pointers and reviews. |
@destogl I can give it a try if you point me into the right direction where to start ;) |
gazebo_ros2_control_demos/urdf/test_gripper_mimic_joint.xacro.urdf
Outdated
Show resolved
Hide resolved
Yes, of course! Your code is great, but I would just like to have a long-term solution to this that is well integrated with other features.
Please check ros-controls/ros2_control#652 for detailed description, and please give your feedback there on how would you like this to be integrated and described in URDF? Since you are using it, you probably have a good idea how to make this nice and straightforward to use in URDF. Then we adjust the parser to it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@christophfroehlich I think we should continue with merging this ros-controls/ros2_control#652 could take few more months, and we should not block this feature for you.
@christophfroehlich I hope I didn't break your idea in #154. Can you comment if that merge changes something here? |
you did ;) But I fixed this with 0d4468a already. I'm not sure if this is an elegant way, but it works at least. I'm not exactly sure what to do with the mimic joints in case of a command_mode_switch. |
Why do you think so? ros-controls/ros2_control#652 needs definitely multiple (breaking) changes so it can take time. If you can or would like to tackle it I can give you pointers where to start. |
Any updates on this? |
Unfortunatley not yet, I never started with ros-controls/ros2_control#652 |
dbc438a
to
d9676a3
Compare
I updated the mimic joint options to be independent on the command interfaces and also added an offset tag like the URDF-cousin has.
See the readme:
Let me know if this is not the intention of the maintainers, but for my robot it was a necessary fix. More elegant would be the usage of transmission tags by ros2_control, but it seems that they are not implemented yet.